test(server-nestjs): verrouille la parité d'émission projectMember + garde anti-consommateur fantôme - #2579
Conversation
496de96 to
b74e256
Compare
yorha-operator
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
Verdict : Approuvé — la PR reste en draft dans l'attente de la décision d'atterrissage.
Delta propre par rapport au socle : +68 lignes de parité projectMember, les trois commits restent dans le périmètre. Ordre d'atterrissage recommandé : #2578 puis celle-ci puis #2583 — le socle partagé restera affiché dans le diff après un atterrissage squash, c'est cosmétique, la fusion reste propre. ✨ Éloge — la garde anti-consommateur fantôme verrouille exactement la classe de régression silencieuse que la migration Fastify→NestJS risque le plus.
yorha-operator
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
Verdict : Approuvé — la PR reste en draft dans l'attente de la décision d'atterrissage.
Delta propre par rapport au socle : +68 lignes de parité projectMember, les trois commits restent dans le périmètre. Ordre d'atterrissage recommandé : #2578 puis celle-ci puis #2583 — le socle partagé restera affiché dans le diff après un atterrissage squash, c'est cosmétique, la fusion reste propre. ✨ Éloge — la garde anti-consommateur fantôme verrouille exactement la classe de régression silencieuse que la migration Fastify→NestJS risque le plus.
…er guard Signed-off-by: William Phetsinorath <[email protected]> Change-Id: Ib2a133a080641583d390f3dbbf7fd5fe6a6a6964
Signed-off-by: William Phetsinorath <[email protected]> Change-Id: Ia0bba6241f9518c234ba6424e68e5f7b6a6a6964
b74e256 to
a2240e5
Compare
|
5 New Issues
0 Fixed Issues
0 Accepted Issues2 participants
Issues liées
#2574
Quel est le comportement actuel ?
project-members.service.tsémetprojectMember.upsert/projectMember.deleteviaAppEventsService, mais aucun consommateur@OnEvent('projectMember.*')n'existe dans server-nestjs — les événements sont actuellement sans effet.Quel est le nouveau comportement ?
project-members.service.spec.ts: +68 lignes — verrouille l'émission aux bons appels (ajout/retrait de membre) et ajoute une garde négative documentée (référence MIGRATION-PARITY-MATRIX.md) qui échouera si un demi-migrateur ajoute un consommateur sans audit de parité.Spec verte sur Node 24.
Cette PR introduit-elle un breaking change ?
Non.
Autres informations
Écart latent documenté (pas de régression live) ; à résoudre au moment du cutover project-member.